Omega V1 Governing Equations Design Document#223
Omega V1 Governing Equations Design Document#223mark-petersen merged 65 commits intoE3SM-Project:developfrom
Conversation
xylar
left a comment
There was a problem hiding this comment.
Some comments, mostly about generalizing the gravitational potential and adding it a few places I think it's missing.
Also, a request to remove the commented-out text to make review easier.
0b73688 to
e70db7a
Compare
c15da50 to
e3573b8
Compare
|
I have added substantial updates based on feedback and conversations this past week. In particular, the document now uses layer mass-thickness |
6d6a68d to
f4f7b14
Compare
6292d53 to
c5aa14b
Compare
| + \nabla_{3D} \cdot \left( \rho {\bf u}_{3D} \otimes {\bf u}_{3D} \right) | ||
| = - \nabla_{3D} p | ||
| + \rho {\bf D}^u + \rho {\bf F}^u | ||
| $$ (continuous-momentum) |
There was a problem hiding this comment.
the buoyancy term is missing here rho g khat
There was a problem hiding this comment.
The first equation set is the most general, so gravity would be within
There was a problem hiding this comment.
oh sorry, totally missed that. Thanks for the explanation
There was a problem hiding this comment.
I realize that one could include gravity in
sbrus89
left a comment
There was a problem hiding this comment.
Thanks for putting this together @mark-petersen, I just have a couple comments.
| \rho = f_{eos}(p,\Theta,S). | ||
| $$ (continuous-eos) | ||
|
|
||
| where conservative temperature, $\Theta$, and salinity, $S$, are examples of tracers $\varphi$. |
There was a problem hiding this comment.
| where conservative temperature, $\Theta$, and salinity, $S$, are examples of tracers $\varphi$. | |
| where conservative temperature, $\Theta$, and absolute salinity, $S_A$, are examples of tracers $\varphi$. |
There was a problem hiding this comment.
Is there a need to work with other types of salinity? If not, I would prefer to just use
fix trailing whitespace Fixes broken cross refs
small change to section formatting Co-authored-by: Hyun (Hyun-Gyu) Kang <47987430+hyungyukang@users.noreply.github.com>
linting fixes updates links
enables numfigs package to allow for figure numbering in design docs
28272b3 to
5d872b5
Compare
|
rebased on head. Squashed small commits about typos, delinting, white space etc, to reduce number of commits. Current compiled version is here: |
|
A negative sign must be included for the pressure terms derived from the surface force terms. According to Leishman 2025, Chapter 21, above equation 2, "the negative sign indicates that the force is inward and in the opposite direction to the unit normal vector area, which always points outward by convention". Also see discussion in Kundu et al. 2016 page 104 and equation 4.26. I added that text to the derivation and changed the sign on the pressure gradient throughout, as well as the related grad(z) terms. |
sbrus89
left a comment
There was a problem hiding this comment.
Approved. Thanks for all the work on this everyone. This is a great resource.
Derivation of non-Boussinesq primitive equations for Omega Version 1.
See compiled document at
https://portal.nersc.gov/project/e3sm/lvroekel/omega_docs/html/design/OmegaV1GoverningEqns.html